Skip to content

refactor: Consolidate Frontent Components and Cleanup#330

Open
piffio wants to merge 2 commits intomainfrom
refactor/consolidate-frontend-components
Open

refactor: Consolidate Frontent Components and Cleanup#330
piffio wants to merge 2 commits intomainfrom
refactor/consolidate-frontend-components

Conversation

@piffio
Copy link
Copy Markdown
Owner

@piffio piffio commented Apr 20, 2026

  • refactor: clean up dead code and reorganize API layer

- Delete unused `$lib/stores/` directory (auth.ts, links.ts)
- Move `fetchUrlTitle()` to API layer (`frontend/src/lib/api/metadata.ts`)
- Move `debounce()` utility to proper location (`frontend/src/lib/utils/helpers.ts`)
- Add `fetchUrlTitle()` method to `linksApi` for cleaner API calls
- Replace `apiClient.request()` with `apiClient.put()` in `linksApi.update()`
- Update `LinkModal.svelte` to use new API locations
- Add `put()` method to `ApiClient` class for consistent API patterns
@piffio piffio changed the title refactor: clean up dead code and reorganize API layer refactor: Consolidate Frontent Components and Cleanup Apr 20, 2026
@piffio piffio added the skip-preview Do not deploy to a preview environment label Apr 20, 2026
@github-actions
Copy link
Copy Markdown

⏭️ Ephemeral Environment Deployment Skipped

Preview deployment was not triggered because:

  • 🏷️ This PR has the skip-preview label

To enable deployment:

  • Remove the skip-preview label

Note: Deployments are also skipped for documentation-only changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-preview Do not deploy to a preview environment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant